Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[dev-v2.7] Add Longhorn 1.5.5 chart #3833

Merged
merged 2 commits into from
Apr 30, 2024

Conversation

mantissahz
Copy link
Contributor

Release Longhorn v1.5.5 into Rancher 2.7

Longhorn longhorn/longhorn#8356
Rancher rancher/rancher#44762

cc @innobead @PhanLe1010 @khushboo-rancher @roger-ryao

Waiting for Longhorn 1.5.5 images merged into the Rancher image-mirror list.

Copy link

Validation steps

  • Ensure all container images have repository and tag on the same level to ensure that all container images are included in rancher-images.txt which are used by airgap customers.
  Ex:-
    longhorn-controller:
      repository: rancher/hardened-sriov-cni
      tag: v2.6.3-build20230913
  
  • Add a 👍 (thumbs up) reaction to this comment once done. CI won't pass without this reaction to the github-action bot's latest validation comment.
  • Approve the PR to run the CI check.

Copy link
Contributor

@PhanLe1010 PhanLe1010 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Could QA help to test @khushboo-rancher ?

release Longhorn v1.5.5 into Rancher 2.7

Longhorn 8356
Rancher 44762

Signed-off-by: James Lu <james.lu@suse.com>
release Longhorn v1.5.5 into Rancher 2.7.

Longhorn 8356
Rancher 44762

Signed-off-by: James Lu <james.lu@suse.com>
Copy link

Validation steps

  • Ensure all container images have repository and tag on the same level to ensure that all container images are included in rancher-images.txt which are used by airgap customers.
  Ex:-
    longhorn-controller:
      repository: rancher/hardened-sriov-cni
      tag: v2.6.3-build20230913
  
  • Add a 👍 (thumbs up) reaction to this comment once done. CI won't pass without this reaction to the github-action bot's latest validation comment.
  • Approve the PR to run the CI check.

Copy link
Contributor

@lucasmlp lucasmlp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@@ -6,31 +6,31 @@
engine:
- repository: longhornio/longhorn-engine
+ repository: rancher/mirrored-longhornio-longhorn-engine
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here the images were patched to use the mirrored ones.

@@ -30,25 +30,25 @@ image:
longhorn:
engine:
repository: longhornio/longhorn-engine
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here the images are not the mirrored ones.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh. This one is in exclude folder which means the whole file will be removed. I think it shouldn't affect anything. WDYT?

@khushboo-rancher
Copy link

Validation - PASSED

Validated with Rancher 2.7.12, installation, uninstallation, upgrade and p0 test cases for volume worked fine. I didn't observe any longhornio images in the deployment, all the components are deployed with mirrored images.

@PhanLe1010 PhanLe1010 merged commit d41bd72 into rancher:dev-v2.7 Apr 30, 2024
11 checks passed
lucasmlp pushed a commit that referenced this pull request May 1, 2024
Signed-off-by: James Lu <james.lu@suse.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants